Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add dev env #8

Merged
merged 60 commits into from
Jan 29, 2025
Merged

Add dev env #8

merged 60 commits into from
Jan 29, 2025

Conversation

RonanMorgan
Copy link
Collaborator

@RonanMorgan RonanMorgan commented Apr 24, 2024 •
edited
Loading

A beautiful dev env which launch a Reoling dev camera and test if an alert has been generated by the engine

PS : I felt in a git hell by trying to remove all .env file in the history so I had to create a new PR

frgfm and others added 21 commits April 3, 2021 16:38
@frgfm
Initial commit
0fca428
@ThibaultDac
Add the base for the docker swarm
ffb11fd
Merge branch 'main' of github.com:pyronear/pyro-devops
5013946
feat: add new docker-compose dev env
93ffa2f 
fix: removal
fix: add user of the same group than devices
1cb8373
feat: add precommit hook config
e8b07b3
feat: add yml precommit
e33c6f6
refactor(init script): use logging lib instead of print
a2df596
feat: add new gitAction for precommit hook
561b2d6 
refactor: rm useless in gitAction precommit
feat: add gitAction to test docker-compose
84c4f94
fix(gitAction docker-compose): add a step to check logs
ef8121e
feat: add healthcheck of pyro-api & change precommit config
c7612da
fix: use python3.9 in gitAction
e879b36
fix: python version error
8a590e9
feat: add a reolink "dev" camera
2814d74
refactor: create a Makefile and new directories
b42c3f4
fix: use Makefile in the gitAction
93a227c
refactor: generate self signed certificate in the reolink script
2602e08
feat: launch test every night
21c87cd
feat: add comment explining how to load images in the env
46e1cf0
feat(test): add a test which will check that an alert has been sent
151b7d7
@RonanMorgan RonanMorgan self-assigned this Apr 24, 2024
@RonanMorgan RonanMorgan marked this pull request as draft April 24, 2024 15:15
@RonanMorgan
Copy link
Collaborator Author

TODO :
=> understand the use case of the media API in order to mimic it in the python test media.py
=> create a centralised server in order to accept the loki call (whitelist some ip in the server itself ? how can we know the ip adress of gitAction servers)

@RonanMorgan
Copy link
Collaborator Author

understand and fix this error :

WARNING:  Could not connect to the endpoint URL: "http://s3.localhost.localstack.cloud:4566/sample-bucket/20240424153150-a7a225e9.jpg"
INFO:     172.18.0.8:45570 - "POST /media/1/upload HTTP/1.1" 500 Internal Server Error
  • we don't see anymore the lines after the grep :
    docker-compose logs --tail=20 | grep -i -E "error|exception" | grep -v -i "hypercorn.error"

Ronan and others added 27 commits May 21, 2024 17:50
30-> 60
b1995a6
feat: add platform container
efa6df3
feat: add a scd camera for test
a629740
feat: update api and init script with new data model
7a1a774
fix: new credentials.json model
86b17f0
feat: add new camera with ptz endpoint
3b7a66f
fix tests
7f480d0
fix: add azimuth in credentials.json
2524663
small fixes
1320562
refactor: rm type from orga
fbb1ce9
rm lat / lon from var env
bae9456
feat: add engine healthcheck
06b05c1
feat: add ETL container
168b215
@RonanMorgan
fix telegram error
e6557f4
@RonanMorgan
small refactor
33c28fe
@RonanMorgan
add file in gitignore
3891f71
@RonanMorgan
rm LOGIN env var from front
9b09b02
@RonanMorgan
fix test detection -> detections
de09c0d
@RonanMorgan
fix test media error
8702372
@RonanMorgan
fix test
d77f8ee
@RonanMorgan
fix precommit hook
c01e57a
@RonanMorgan
fix CI
1e5069e
@RonanMorgan
Readme. Rm useless var env
a88b5c4
@RonanMorgan
README
c9bda0b
@RonanMorgan
try fixing CI
0ad605a
@RonanMorgan
fix github workflow
e47b802
@RonanMorgan
fix CI
8627b32
MateoLostanlen
MateoLostanlen approved these changes Jan 29, 2025
View reviewed changes
Copy link
Member

@MateoLostanlen MateoLostanlen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for this PR, it wiil be really helfull for all Pyronear volonteers !

README.md
https://docs.docker.com/engine/swarm/swarm-tutorial/create-swarm/
Start the Docker services using the following command:
```
make build
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please make this on two lines or add a && I don't know why but it's display one line

@MateoLostanlen MateoLostanlen merged commit a61c83d into main Jan 29, 2025
2 checks passed
@RonanMorgan RonanMorgan deleted the rs/add-dev-env branch January 29, 2025 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MateoLostanlen MateoLostanlen MateoLostanlen approved these changes

@frgfm frgfm Awaiting requested review from frgfm

Assignees

@RonanMorgan RonanMorgan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@RonanMorgan @MateoLostanlen @frgfm @ThibaultDac